Skip to content

Add pkg-config file#305

Open
panicgh wants to merge 1 commit intoeProsima:masterfrom
panicgh:pkgconfig-file
Open

Add pkg-config file#305
panicgh wants to merge 1 commit intoeProsima:masterfrom
panicgh:pkgconfig-file

Conversation

@panicgh
Copy link
Copy Markdown

@panicgh panicgh commented Apr 25, 2026

Description

Currently, Fast-CDR only integrates with CMake, but there are many projects out there that are not built with CMake and thus cannot directly include Fast-CDR. pkg-config is often the least common denominator by many build systems.

Generate and install a pkg-config file to make Fast-CDR also usable by other build systems.

I checked the CMake config file for the FASTCDR_DYN_LINK and FASTCDR_STATIC_LINK flags and had to do a bit of a dance in pkg-config, but I hope -DFASTCDR_DYN_LINK -UFASTCDR_DYN_LINK -DFASTCDR_STATIC_LINK works satisfactorily.

Contributor Checklist

  • Commit messages follow the project guidelines.
  • The code follows the style guidelines of this project.
  • N/A Tests that thoroughly check the new feature have been added/Regression tests checking the bug and its fix have been added; the added tests pass locally
  • N/A Any new/modified methods have been properly documented using Doxygen.
  • Changes are backport compatible: they do NOT break ABI nor change library core behavior.
  • Changes are API compatible.
  • N/A New feature has been added to the versions.md file (if applicable).
  • N/A Applicable backports have been included in the description.

Reviewer Checklist

  • The PR has a milestone assigned.
  • The title and description correctly express the PR's purpose.
  • Check contributor checklist is correct.
  • Check CI results: changes do not issue any warning.
  • Check CI results: CI pass and failing tests are unrelated with the changes.

Signed-off-by: Nicolas Benes <nbenes.gh@xandea.de>
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

1 participant